Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes for Viash 0.9 #27

Merged
merged 51 commits into from
Jun 28, 2024
Merged

Fixes for Viash 0.9 #27

merged 51 commits into from
Jun 28, 2024

Conversation

rcannood
Copy link
Member

@rcannood rcannood commented Feb 24, 2024

viash-actions v6.0.0

Breaking changes

  • update-docker-engine was moved to project/update-docker-engine.

  • pro/build-nextflow-params arguments components and workflows were removed in favour for the target_dir argument.

  • pro/build-nextflow-schemas arguments components and workflows were removed in favour for the target_dir argument.

New functionality

  • project/build-target: Created an action for building a target directory using viash ns build and generating a matrix of the components for which Docker images need to be built and pushed.

  • project/check-concurrent-pr: Created an action for checking if a push event is concurrent with a pull request.

Major changes

  • All actions were updated to work with Viash 0.9 and later.

Documentation

  • Updated all READMEs to be more consistent.

  • Added a Makefile for generating READMEs more efficiently.

@rcannood rcannood changed the title Implement try catch strategy for viash 0.9 Fixes for Viash 0.9 Feb 25, 2024
@rcannood rcannood requested a review from DriesSchaumont June 27, 2024 10:41
Copy link
Contributor

@DriesSchaumont DriesSchaumont left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small remark, LGTM otherwise!

CHANGELOG.md Show resolved Hide resolved
@rcannood rcannood merged commit 9706ce0 into main Jun 28, 2024
@rcannood rcannood deleted the viash-0.9 branch June 28, 2024 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants